bump qt build from 6.5.2 to 6.5.3 (#1185)

Remove final remnants of 'exported' (#1146)

* Remove final remnants of 'exported'

* incorporate review comments in duplicate filter,

and remove deleted exported keyword from unicsv doc.

---------

Co-authored-by: Robert Lipe <robertlipe@gmail.com>

tidy fixes (#1179)

* tidy readability-convert-member-functions-to-static

* tidy readability-make-member-function-const

* kill trailing space

fix various "Redundancies in code"  (#1178)

* fix "Redundant elaborted type specifier"

* fix "Redundant 'inline' specifier"

* fix "redundant 'const' specifier on a constexpr variable"

* remove "Redundant qualifier"

* add /out/, /.vs/ to .gitignore, and sort.

fix SchemaArrayField/SchemaArrayData mismatch (#1177)

* fix SchemaArrayField/SchemaArrayData mismatch

related to IGC additions from #1060 and #1054.

* retire global trait collection

* use class member to store track traits instead of fs data.

* fiddle with kml accumlate hash

* compute number of wp_fields

* correct bug concerning track specific traits.

* simplify multi-track array decisions.

* simplify kml multi-track schema decisions.

Integrate OAT and SIU with Waypoints (#1060)

* Integrate OAT and SIU with Waypoints

* Type corrections, convert set_value() overload to nullptr* check

* #1062 Fix core dump when merging tracks with disparate IGC extensions

* Reduce `value` variable scope when writing SimpleArrays

* Debug oopsie bugfix, fix core dump when merging IGC files with no extensions

* Collapse redundant if statement

* Fix brain fart

* Eliminate igc_fsdata::has_value()

* Fix subtle satellite count bug

* Fix small sat count bug, and eliminate some ifdefs

* Make some IGC exts override default WP members

* Include IGC line-by-line debug info

* Remove accidentally included file

* delete obsolete macros

---------

Co-authored-by: tsteven4 <13596209+tsteven4@users.noreply.github.com>

fix noreturn booboos. (#1176)

* fix warning: function declared ‘noreturn’ has a ‘return’ statement

* geez, another booboo using noreturn.

Merge pull request #1175 from tsteven4/refact

Apply new resharper refactoring "Redundant dereferencing and taking address"

fix reserved-macro-identifier clang diagnostic

use noreturn attribute

drop extra semicolons

don't use posix but non-iso argument reordering in print specifiers.

remove obsolete declarations

Apply new resharper refactoring "Redundant dereferencing and taking address"

make all but the stack filter dynamic. (#1174)

Bump actions/checkout from 3 to 4 (#1172)

Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](https://github.com/actions/checkout/compare/v3...v4)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: tsteven4 <13596209+tsteven4@users.noreply.github.com>

mkshort becomes a class (#1173)

* make mkshort a class

* add new file mkshort.h

* refactor names

* use bools with mkshort setters

convert mkshort from char* to QByteArray (#1171)

* convert mkshort from char* to QByteArray.

and store conflict count directly in hash.

* fix potential detach

and rename vars

* tweak whitespace removal.

* don't split codepoints that comprise a grapheme.

* enhance mkshort test

* fix enhanced test

* test mkshort replacements

* remformat mkshort with astyle

* add mkshort test to cmake

Update gpsbabel_man.xml (#1170)

Without a dash this will resemble the "tar" command!

fix mkshort unique (#1168)

* fix mkshort unique.

1. we were off by one when deciding if we could just concatentate
the generated suffix to the name, failing to use the last available
character.
2. when attempting to copy the generated suffix to a presumably
shortened name, and the generated suffix was longer than the name,
the target of the copy was outside the name buffer.  In the common
case that the rank of size_t was greater than the rank of int, the
target was well beyond the end of name, not before as one would expect
with signed arithmetic.

With the new algorithm when the target length is insufficient to
fit both the name and the suffix we will only truncate the name as
required to fit the truncated name and the complete suffix in the
target length(as opposed to the original length of the name).
We fatal if this is not possible.

Add a test case to exercise the make unique code.

* fix reference mode.

* add notes for future enhancements

* fix testcase cut and paste booboo

* refactor mkshort input from char* to QByteArray.

convert googletakeout format to dynamic. (#1167)

delete obsolte and unused macro. (#1166)

delete undefined declaration. (#1165)

The declaration appeared in 6a9a05864, 7/25/2011, without a definition
or usage.

fix memory leak in mkshort from d7dfed6, 1/23/2005. (#1162)

Add support for Google Takeout Location History (#1160)

* Add support for Google Takeout Location History

This adds support for parsing the location history provided by Google Takeout.
This makes it possible to convert a month's, year's, or your entire location
history at once instead of having to export one day's history at a time on
Google Maps.

There is documentation in the `xmldoc` folder with some examples.

It works great, but there are one, possibly two, other additions I'd like to
make in following diffs:

1. Add the ability to select date ranges. For example, I went on a road trip
   that started in the last week of February, but right now I can only select
 tracks at the resolution of a whole month.

2. Add support for the higher-resolution "roadSegment" blocks in newer
   "activitySegment" blocks. This will have to be optional as you have to
 query the Google Maps API to get lat/long, which costs money. However,
 if you're willing to pay for it, you can get great detail out of it.
 They look like this:

```
       "roadSegment": [{
          "placeId": "ChIJBdXmuuWaK4gRoiClERhZcy8",
          "duration": "76s"
        }, {
          "placeId": "ChIJ4ekNlOWaK4gRxu-moMRFo60",
          "duration": "58s"
        }, {
          "placeId": "ChIJ10qEheWaK4gRQ1T24wEuuN4",
          "duration": "58s"
        }, {
          "placeId": "ChIJe7NWfu-aK4gRFHBB6P0PMe4",
          "duration": "58s"
[...]
```

* rename variable

* this TODO is done

* now builds on focal, fix documentation

* patch -p2 < ~/testo.log

* use `&&` instead of `and` for old compilers

* don't leak objects when we skip events at exact lat/lon 0/0

* use a reference (even though the compiler would have optimized that away)

* rename to googletakeout

* fix docs

* Address code review requests

* add `googletakeout.h` to HEADERS
* convert most `#define`s to static members
* don't pollute global namespace
* rename logging functions
* convert some methods to static functions for clang-tidy
* `title_case` loop by reference instead of by index (clang-tidy)
* better use of debugging levels
* drop dead simplifiedRawPath code for now
* drop empty tracks
* default constructor for GoogleTakeoutInputStream
* add some tests that check the content of the data

Also, I found some unofficial documentation about the location history
format and added that (thanks @CarlosBergillos !)

* apply @tsteven4 's patch. thank you!!

* add license

* fix logging

* As I assigned the correct license to my code, I am skeptical that this makes a difference, but give Robert co-copyright.

* make the test more boring

* rm xml_slice

* revert constructor change from @tsteven4 's patch

* apply @tsteven4 's latest constructor

* apply @tsteven4 's latest patch -- `cmake --build . --target check` passes

---------

Co-authored-by: tsteven4 <13596209+tsteven4@users.noreply.github.com>

retire strupper, strlower. (#1164)

kill tpo2 write code, which has been dead since 46676d105, 5/17/2006. (#1163)

introduce clang-tidy to CI and integrate with codacy (#1071)

* add codacy clang-tidy to workflows

* attempt to run clang-tidy on ci

* update jammy image.

get rid of ppa
add clang-tidy, jq.

* fixes.

* whittle down tidy checks.

* kick out another tidy warning

resolve maintanence issue in interpolate & resample filters. (#1161)

* resolve maintanence issue in interpolate filter.

Instead of manually creating a deep copy of the route list with
an empty waypoint list we operate on the original route list by
swapping it's waypoint list with an empty list.  This is both more
efficient and easier to maintain.

* resolve maintanence issues in resample filter.

Instead of manually creating a deep copy of the route list with
an empty waypoint list we operate on the original route list by
swapping it's waypoint list with an empty list.  This is both more
efficient and easier to maintain.

For efficient deletion use track_del_marked_wpts.

remove obsolete files (#1158)

* remove obsolete files

* delete some more obsolete files

adjust greatcircle conversion factors, kill gcgeodist cheat. (#1156)

* tweak greatcircle conversions.

* modify arc-project test to use microdegrees instead of nanodegrees.

* kill unused metric prefix conversion macros.

remove erroneous addition from #1151. (#1157)

The need for this statement was eliminated with the introduction of
wpt_flags.marked_for_deletion.

use efficient methods to delete trkpts in track filter. (#1155)

* use efficient methods to delete trkpts in track filter.

* clean up some code issues flagged by windows tools.

fix interator invalidation issue with trackfilter segment option. (#1153)

and add a test case that demonstrates the issue.

use std::prev instead of --c.end() (#1154)

--c.end() is not guaranteed to compile.

Introduce efficient member functions for deleting waypoints (#1151)

* Introduce efficient member functions for deleting waypoints

New functions with complexity O(n) for deleting pre-marked
waypoints are:
  WaypointList::del_wpts
  RouteList::del_wpts
  del_wpts
  route_del_wpts
  track_del_wpts

Use those functions instead of the inefficient versions
WaypointList::waypt_del
RouteList::del_wpt
waypt_del
route_del_wpt
track_del_wpt.
When these functions are using while looping over a waypoint list
the overall complexity is O(n^2).  This is because these functions
themselves amortize to O(n).

* review nits.

* tweak deletes

* use dedicated wpt deletion flag.

Reduce complexity of simplify filter (#1149)

* fix bugs with simplify filter

1. With the length option the last point deleted took the total error
over the specified limit.

2. When computing the total error if another point is deleted it was
possible to refer to an xte record that needed to be updated due to
the deletion of one of its neighbors.

* wip on reduced complexity simplify filter.

* mimic ineheitence of new_trkseg flag in simplify filter.

* cleanup simplify header.

* work around apparent Qt5 QMap bug.

* fix shadowing issues.

* astyle

* correct spelling.

* kill Waypoint route_priority.

time read, filter and writer operations (#1150)

* time read, filter and writer operations

when using -D > 0.

* take stray dog to the pound.

fix bugs with simplify filter (#1148)

1. With the length option the last point deleted took the total error
over the specified limit.

2. When computing the total error if another point is deleted it was
possible to refer to an xte record that needed to be updated due to
the deletion of one of its neighbors.

improve correctness and speed of duplicate filter. (#1144)

* improve correctness and speed of duplicate filter.

The duplicate filter could errouneously delete points that were
not duplicates if the crc's happened to match.

waypt_del(Waypoint*) is inefficent as it requires a search of the
list to find the matching waypoint.  Support waypt_del with iterators.

* retire util_crc.cc

* improve duplicate to linear complexity

* polish new list creation.

* Remove final remnants of 'exported'

* Revert "Remove final remnants of 'exported'"

This reverts commit 6996e4d7dd4b1812f922edd2c5bd9a3d1bc6fada.

---------

Co-authored-by: Robert Lipe <robertlipe@gmail.com>

update Qt 6.5.1 -> 6.5.2 in CI. (#1140)

sync test case list in CMakeLists.txt with testo.d. (#1143)

Yes, manual syncing is suboptimal.

fix mkshort qHash return type. (#1142)

reduce complexity of gdb waypt searchs from O(n^2) to O(n) (#1141)

* use QHash for gdb waypoint searches.

This reduces the complexity from O(n^2) to O(n).

For the reader two parallel hashes are maintained with different
keys but identical values.  This is necessary because sometimes
we want to match the name and position, and other times we just want
to match the name.

* workaround missing qHashMulti in Qt5.

and supply missing default for our qHash functions.

* fix MSVC C2666 errors due to an erroneous assumption.

In Qt6 the return type of qHash, and the type of the seed,
happens to be the same as QHash::size_type.
However, this isn't true in Qt5.

reduce complexity of mkshort from O(n^2) to O(n). (#1139)

* reduce complexity of mkshort from O(n^2) to O(n).

* add converting constructor for mkshort hash key

Deprecate navilink, sbn, sbp. (#1138)

As discussed in
https://github.com/GPSBabel/gpsbabel/pull/1090
these units have long since been deprecated by their maker. We have no
known remaining users. Numerous attempts (including  phone calls and
attempt to contact via the windsurfing community) to reach the last
known users have all failed.

Buh Bye.

deprecate wbt formats. (#1136)

eliminate ppa for qlibraryinfo bug (#1135)

Strptime update (#1133)

* prepare to update strptime

* update strptime to glibc 2.37, with modifications.

* add missing cmake file.

* quiet MSVC warnings with strptime

* quiet codacy complaints about strptime

third party code

* use _strnicmp on windows

* ensure strptime works with signed enums for windows.

fix clang 15 warnings on windows (#1132)

* fix some format specifier issues

that came to light trying to get clang to work under visual studio.

* fix another windows only warning

* fix some more wordsize issues.

add gnu attribute to check functions with printf like parameters. (#1129)

and fix a bug revealed by the warning.

fix gdb debug (#1128)

* fix gdb dbg.

* use bool literals

fix humminbird icon matching, (#1126)

* fix humminbird icon matching,

eliminating one use of xasprintf.
This was broken in 6029c8267 over a decade ago.

* eliminate unnecessary xasprinf usage.

* eliminate another xasprintf usage.

* eliminate xasprintf usage.

* retire xasprintf, xvasprintf.

drop support for filesystem access via garmin usb. (#1127)

add fedora 38 to CI (#1125)

fix nits with trackfilter (#1124)

* fix nits with trackfilter

* deprecate mapconverter readme.

mapconverter style file was removed in #870

* catch reference files up

switch some builds to Qt 6.5.1 (#1119)

* update some builds to Qt 6.5.1

* conditional code for QVersionNumber evolution

fix garmin serial/usb device character encoding/decoding. (#1117)

* attempt to use correct codec with garmin reader.

the garmin writer is unchanged as of yet.

* fix garmin writer wrt encoding.

* add garmin option to force codec.

* update serialization ref files for garmin codec option.

* fix potential overwrite bug with route/track names

in garmin writer.

* fix up for old Qt.

* correct assertion

* use static_assert to for fixed buffer size checks.

* document garmin codec option

* don't clean rtept/trkpt names for newer devices.

We clear badchars for newer devices already, effectively not
cleaning wpt names.

refactor unicsv & xcsv date time handling (#1114)

* refactor unicsv date time handling.

* restrict utc option range to match Qt offsetfromutc

* move xcsv from C-style legacy time to Qt.

* fill in lower order date/time fields when parsing.

* don't return date/time if we don't have one!

* enhance xcsv date/time testing, fix bug.

* csv format date time adjustements.

use QDateTime::fromString to parse iso date times.
return invalid QTime from addhms if parsing fails entirely.

* datetime display fixes.

For the xcsv writer:
avoid priting date/time fields when there isn't a valid date/time.
when printing with am/pm use times from a 12 hour clock.
add support for printing dotnet time.
use QDateTime::toString to print ISO time.
For the xcsv and unicsv readers:
use QDateTime::fromString instead of xml_parse_time when reading
ISO datetimes.  This avoids xml_parse_times intentional odd behavior
of treating non-timezoned times as UTC.

add a test of the xcsv writer time related fields.  This only runs
in America/Denver time zone.  As setting the time zone is system
dependent the test only runs if tzselect is available.  If so it
assumes America/Denver is available.

correct documentation to give a sensible format for xcsv field GMT_TIME.

* add reference files for new test.

* silence xcsv reader conversion warnings on empty strings.

* warn on parse errors reading excel time.

* xcsv date/time fixes.

fix addhms to account for 12/24 hour clock.
don't print invalid datetimes with iso_time, iso_time_ms.

add testcases to exercise all the xcsv reader date/time flavors.

* clarify HMSL, HMSG wrt 12/24 hour clocks.

* add missing reference file.

* enhance xcsv for HMS[L|G] before or after [LOCAL|GMT]_TIME

* restore YYYYYMMDD to use UTC.

This has been broken for some time.  Mail from 2012 indicates the intent
was UTC (https://sourceforge.net/p/gpsbabel/mailman/message/29544538/)

* unicsv review catches

don't ripple into trouble with msec rounding.
pass outputs that may or may not be written as references.

* retire xcsv fields HMSG_TIME, HMSL_TIME.

This is potentially a user visible change.  It could require
users to rewrite any style files they have created that use
these fields.

These are replaced by repeated use of GMT_TIME and LOCAL_TIME.
This eliminates our pain over 12/24 hour clock issues. strptime/
strftime (as well as QTime) have distinct conversion specifiers
for hours with 12/24 hour clocks.  Our support for HMSG_TIME,
HMSL_TIME used integer conversion specifiers for hours, minutes
and seconds.  This made it difficult to decide if a 12 or 24 hour
clock should be used, and made it impossible to have reduced
precision values with 12 hour clock using an AM/PM designation.
We also always printed AM/PM designations.

* fix whitespace in serialization reference file

restore tab when display vector formats. (#1116)

this was lost in #1068
update reference files to be exactly what the test produces, including
exact whitespace, trailing tabs and no newline at end of file.

add testing for kml:dateTimeType. (#1112)

and fix a related bug.

Note that we incorrectly, for kml:dateTimeType, force values
without time zone qualifiers to UTC.
We do not test the optional gYearMonth time zone qualifer.
We do not test the optional gYear time zone qualifer.
We do not test the optional date time zone qualifer.

workaound curl bug in lastest windows github runner. (#1111)

update linux snap (#1110)

* switch snap from devmode to strict.

* date first in version string

* update snap: strict, no gui, layout

create a snap of the cli (#1108)

* try to build a snap on github.

* save artifact

* add test and continuous release

* permissions

* try to get sha into snap

* try harder to get sha in

* use pre-existing source

* debug missing sha

* force version into snap instead of sha

* shorten version

* make version more sortable

* avoid dying on info display

* pipes vs ors

* more work on sha

* again sha...

restrict osm timestamps. (#1107)

osm consumers may not accept fractional seconds, but all should
support whole seconds.
osm consumers may not accept time zone designations other than Z,
but all should support time zone designation Z.
See https://github.com/osmcode/osmium-tool/issues/267

datetime purity. (#1102)

eliminate some QDateTime -> gpsbabel:DateTime conversions.  These
can also help performance as DateTime defaults to UTC.

eliminate some unecessary involvment of the UNIX epoch when computing
time differences.

use QRegularExpression for xml generic tag matching. (#1099)

* use QRegularExpression for xml generic tag matching.

The kml reader now creates tracks from innerBoundryIs elements in
addition to the historic creation from outerBoundryIs elements.

* collapse kml 2.3 track elements

* const correctness improvements for xmlgeneric.

remove C style legacy time from sbn format. (#1096)

remove remnants of baroiq format. (#1095)

aka brauniger_iq.

set timespec to UTC in new skytraq code. (#1094)

I wish Qt would default to UTC instead of Local Time!

move skytraq away from C style legacy time. (#1092)

Co-authored-by: Robert Lipe <robertlipe@users.noreply.github.com>

finish removing C style legacy time from navilink. (#1093)

coverage is provided by sbp test.

move navilink towards QDateTime (#1091)

* enable navilink waypoint write test.

add rounding for round trip fidelity.
the track test has more serious mismatches.

* move navilink towards QDateTime.

reading the datalog is untested and still uses the legacy C-style
date and time library.

* use binary mode in navilink writer.

C-style date and time -> QDateTime in wbt format. (#1090)

* test wbt timestamps.

* C-style date and time -> QDateTime in wbt format.

use Qt datetime when writing gdb header. (#1089)

* use Qt datetime when writing gdb header.

* fiddle with comment on gdb header history.

add gnu attributes to addtional printf like functions (#1088)

* add gnu attribute for format string checks on additional functions.

* silence msvc whining about unknown attributes.

In c++17 unknown attributes should be ignored without causing an error.
But these warnings are excessive:
"warning C5030: attribute 'gnu::format' is not recognized"

add build date from CI. (#1087)

reorg trackfilter.h (#1086)

and fix some related
cppcoreguidelines-macro-usage
modernize-return-braced-init-list

hide inifile_t from users (#1085)

* hide inifile_t from users.

* explicitly make vars have internal linkage

although since c++11 all vars declared in unnamed namespaces will
anyway.

Forgot "name" (#1081)

Or maybe use capitals:
"Latitude, Longitude, Name"

I am guessing that Name is correct.

Co-authored-by: Robert Lipe <robertlipe@users.noreply.github.com>

-x arc: Mention output is reordered! (#1082)

The output isn't simply filtered, but also reordered!

modernize garmin_txt (#1080)

* modernize garmin_txt string handling.

also:
wean from strupper.
test binding fields to the headers in the reader.
test the writer.

* demacro garmin_txt

* eliminate redundant assignments

* astyle

* use containers for garmin_txt mapping info.

* garmin_txt reader warn on unrecognized header

and retire memset.

* use some bools, change parse_date_and_time signature.

fix some readability-implict-bool-conversion tidy warnings (#1072)

* fix some readability-implicit-bool-conversion

warnings from clang-tidy

* another implicit bool

* fix tidy modernize-use-auto

* fix google-readability-namespace-comments

struct tm -> std::tm, memset of tm goes to {} (#1079)

* struct tm -> std::tm, memset of tm goes to {}.

* init std::tm related variables.

tidy found found the following warnigns related to our usage of
std::tm:
cppcoreguidelines-pro-type-member-init
and std::tm*:
cppcoreguidelines-init-variables

* make GPS_D600_Get var tm automatic storage duration

and intialize it.

gb_[set|get]bit -> std::bitset (#1078)

fix kml cppcheck warnings. (#1077)

Operator '|' with one operand equal to zero is redundant.

fix some MISRA 20.7 warnings (#1076)

* fix some cppcheck warnings MISRA 20.7

and use QStringView in gdb writer routines.

* migrate gdb macros to static constexpr.

* name gdb constants per google style guidelines.

keep gdb route point names and waypoint names aligned. (#1075)

This used to work in 1.4.4, but stopped working with the conversion
to QString (NEW_STRINGS).

Add a test case to verify the names match.

introduce route_head::empty() (#1074)

exercise qt 6.5.0 on macos CI (#1073)

fix cppcheck constVariable warnings (#1070)

* fix cppcheck constVariable warnings.

* guard against qt container detachement.

use c++ style attribute syntax with gnu namespace. (#1069)

* use c++ style attribute syntax with gnu namespace.

cppcheck 2.10.3 seems to have difficulty combining c++ style
attributes like [[noreturn]] with gnu __attribute__ syntax.
this leads to lots of false positive [nullPointerRedundantCheck]
warings.

I have a feeling I did this before and it failed somewhere.  Perhaps
our compilers are all ready for it now.

* fix 7 real nullPointerRedundantCheck warnings.

and
7 nullPointerArithmeticRedundantCheck warnings
1 redundantInitialization warning
1 unreadVariable warning

* fix 1 nullPointerRedundantCheck.

A more involved fix would be to pass a reference to a non-const
parameter to mkshort_del_handle to be used as an in-out parameter.

* fix 1 nullPointerRedundantCheck

* workaround cppcheck bug with noretun.

de-duplicate format and filter vector display code. (#1068)

* de-duplicate format and filter vector display code.

* consolidate release of argument memory in vecs, filter_vecs.

add support for dynamic filters (#1008)

* support dynamic filters.

* delete static ResampleFilter.

* make some more filters dynamic.

* parrot Vecs::find_vec in FilterVecs::find_filter_vec.

---------

Co-authored-by: Robert Lipe <robertlipe@users.noreply.github.com>

use constexpr if for trackfilter debug (#1067)

Co-authored-by: Robert Lipe <robertlipe@users.noreply.github.com>

reference/paris.wpo was part of Holux-gm100 and no longer needed.

Remove holux gm-100 format. (#1065)

Following through with plan announced Feb 28, 2022.

This was a serial unit from the turn of the century. Holux is out of business.
We've done nothing but mechanical maintenance on this format for years.

Co-authored-by: Robert Lipe <robertlipe@users.noreply.github.com>

Removing baroiq and cambridge formats. (#1066)

* Remove Cambridge and Baroiq formats as per Feb 28 RFC.

Make locale debugging a compile-time constexpr debug instead of #ifdef. (#1011)

* Make locale debugging a compile-time constexpr debug instead of #ifdef.

* Ignore *.a in git so a branch change work more effortlessly on MacOS. (Yeah, not part of this CL...)

Co-authored-by: GPSBabel <12013583+GPSBabelDeveloper@users.noreply.github.com>

Update interpolate-time.xml (#1021)

Important to explicitly show fractional seconds are OK.

Co-authored-by: Robert Lipe <robertlipe@users.noreply.github.com>